da656a5492d9e5b3c3a8d281ecc08864fa1f5c83,src/main/java/org/olat/ims/qti21/ui/editor/ItemSessionControlController.java,ItemSessionControlController,formOK,#UserRequest#,222
Before Change
itemSessionControl.setAllowComment(Boolean.FALSE);
}
if(allowReviewEl.isOneSelected() && allowReviewEl.isSelected(0)) {
checkNotNull(itemSessionControl).setAllowReview(Boolean.TRUE);
} else if(itemSessionControl != null) {
itemSessionControl.setAllowReview(Boolean.FALSE);
}
if(showSolutionEl.isOneSelected() && showSolutionEl.isSelected(0)) {
After Change
@Override
protected void formOK(UserRequest ureq) {
ItemSessionControl itemSessionControl = part.getItemSessionControl();//can be null
// need to be first!
if(allowSkippingEl.isOneSelected()) {
if(allowSkippingEl.isSelected(0)) {//yes
checkNotNull(itemSessionControl).setAllowSkipping(Boolean.TRUE);
} else if(allowSkippingEl.isSelected(1)) {//no
checkNotNull(itemSessionControl).setAllowSkipping(Boolean.FALSE);
} else if(itemSessionControl != null) {//inherit
itemSessionControl.setAllowSkipping(null);
}
}
if(allowCommentEl.isOneSelected()) {
if(allowCommentEl.isSelected(0)) {
checkNotNull(itemSessionControl).setAllowComment(Boolean.TRUE);
} else if(allowCommentEl.isSelected(1)) {
checkNotNull(itemSessionControl).setAllowComment(Boolean.FALSE);
} else if(itemSessionControl != null) {
itemSessionControl.setAllowComment(null);
}
}
if(allowReviewEl.isOneSelected()) {
if(allowReviewEl.isSelected(0)) {
checkNotNull(itemSessionControl).setAllowReview(Boolean.TRUE);
} else if(allowReviewEl.isSelected(1)) {
checkNotNull(itemSessionControl).setAllowReview(Boolean.FALSE);
} else if(itemSessionControl != null) {
itemSessionControl.setAllowReview(null);
}